Skip to content

add pyproject.toml for pip-installable package#9

Merged
kaituoxu merged 3 commits intoFireRedTeam:mainfrom
eschmidbauer:main
Mar 9, 2026
Merged

add pyproject.toml for pip-installable package#9
kaituoxu merged 3 commits intoFireRedTeam:mainfrom
eschmidbauer:main

Conversation

@eschmidbauer
Copy link
Copy Markdown
Contributor

No description provided.

@kaituoxu
Copy link
Copy Markdown
Collaborator

kaituoxu commented Feb 22, 2026

‎pyproject.tom dependecy part, can you add version limitation? see version in requirements.txt

@eschmidbauer
Copy link
Copy Markdown
Contributor Author

@kaituoxu done.

@kaituoxu
Copy link
Copy Markdown
Collaborator

Can pyproject.tom use "==" but not ">="? and use "--extra-index-url https://download.pytorch.org/whl/cu118"?

because I only test == versions, not test >= versions.

@eschmidbauer
Copy link
Copy Markdown
Contributor Author

Can pyproject.tom use "==" but not ">="? and use "--extra-index-url https://download.pytorch.org/whl/cu118"?

because I only test == versions, not test >= versions.

Done.

@kaituoxu
Copy link
Copy Markdown
Collaborator

torch==2.1.0+cu118
torchaudio==2.1.0+cu118
transformers==4.51.3
numpy==1.26.1
cn2an==0.5.23
kaldiio==2.18.0
kaldi_native_fbank==1.15
sentencepiece==0.1.99
soundfile==0.12.1
textgrid

can you use this version?

@eschmidbauer
Copy link
Copy Markdown
Contributor Author

torch==2.1.0+cu118
torchaudio==2.1.0+cu118
transformers==4.51.3
numpy==1.26.1
cn2an==0.5.23
kaldiio==2.18.0
kaldi_native_fbank==1.15
sentencepiece==0.1.99
soundfile==0.12.1
textgrid

can you use this version?

these versions do not work in python 3.12 so we would have to force python 3.11

@eschmidbauer
Copy link
Copy Markdown
Contributor Author

i tested the current pyproject.toml in this PR with python 3.11 and python 3.12
is there a reason you want to run those older packages?

@kaituoxu
Copy link
Copy Markdown
Collaborator

ok, get.
Thanks for your PR!

@kaituoxu kaituoxu merged commit f12ba28 into FireRedTeam:main Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants